Skip to content

Add _common submodules to OTLP exporter docs (Fixes #2573)#5175

Open
64johnlee wants to merge 2 commits intoopen-telemetry:mainfrom
64johnlee:ralph/otel-2573-docs
Open

Add _common submodules to OTLP exporter docs (Fixes #2573)#5175
64johnlee wants to merge 2 commits intoopen-telemetry:mainfrom
64johnlee:ralph/otel-2573-docs

Conversation

@64johnlee
Copy link
Copy Markdown

Summary

Fixes #2573.

Adds :automodule: entries for the _common submodules in both the HTTP and gRPC OTLP exporter documentation, so that all exported modules are documented in the Sphinx docs.

Changes

In docs/exporter/otlp/otlp.rst:

  • Added documentation for opentelemetry.exporter.otlp.proto.http._common
  • Added documentation for opentelemetry.exporter.otlp.proto.grpc._common

Both now include the standard :members:, :undoc-members:, and :show-inheritance: directives.

64johnlee added 2 commits May 5, 2026 08:33
The automodule directives for trace_exporter, metric_exporter, and
_log_exporter submodules in both grpc and http were missing
:members: :undoc-members: :show-inheritance:, so the API docs
for these weren't being generated.
Fixes open-telemetry#2786.
Fixes open-telemetry#2573

Adds :automodule: entries for the _common submodules in both the
HTTP and gRPC OTLP exporter documentation, so that all exported
modules are documented in the Sphinx docs.
@64johnlee 64johnlee requested a review from a team as a code owner May 5, 2026 01:55
@linux-foundation-easycla
Copy link
Copy Markdown

CLA Missing ID CLA Not Signed

@tammy-baylis-swi
Copy link
Copy Markdown
Contributor

Thanks @64johnlee ! Please sign the CLA for this one too.

I think this will conflict with #5124 which has already been approved, so maybe wait for that to merge first.

@tammy-baylis-swi tammy-baylis-swi moved this to Reviewed PRs that need fixes in Python PR digest May 5, 2026
@64johnlee
Copy link
Copy Markdown
Author

I have signed the CLA. Please re-check the EasyCLA status.

@64johnlee
Copy link
Copy Markdown
Author

CLA signed and email verified. Please re-check EasyCLA status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Reviewed PRs that need fixes

Development

Successfully merging this pull request may close these issues.

Some documentation not built into sphinx docs

2 participants